Skip to content

Acknowledge OSCUR funding via readme — inspired by @jonfroehlich#73

Merged
soniacq merged 1 commit intomainfrom
refactor/OSCUR_funding
Aug 21, 2025
Merged

Acknowledge OSCUR funding via readme — inspired by @jonfroehlich#73
soniacq merged 1 commit intomainfrom
refactor/OSCUR_funding

Conversation

@simonprovost
Copy link
Member

@simonprovost simonprovost commented Aug 15, 2025

Hi team!

  1. OSCUR clean logo added as an asset
  2. Update readme with OSCUR logo for funding acknowledgement, inspired by https://github.com/ProjectSidewalk/RampNet
    w/ @jonfroehlich

Viz. Preview:

Screenshot 2025-08-15 at 2 11 31 PM

Cheers

@simonprovost simonprovost requested a review from soniacq August 15, 2025 13:09
@simonprovost simonprovost self-assigned this Aug 15, 2025
@simonprovost simonprovost added the documentation Improvements or additions to documentation label Aug 15, 2025
Copy link
Contributor

@soniacq soniacq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding an acknowledgment is a great idea. However, the icon appears too large; I suggest reducing it to 250 px or even 200 px.

@jonfroehlich
Copy link

That's what we did:

Code:

## Acknowledgments
This work is supported by the NSF and is part of the [OSCUR initiative]([url](https://oscur.org/)).

<img src="assets/oscur_logo_black.png" width="300px">
image

@simonprovost
Copy link
Member Author

simonprovost commented Aug 15, 2025

Ill wait for the final call by @soniacq prior to doing any change, given that @jonfroehlich kindly answered!

IMHO: I have no preference at all; choose a px and I'll push the change. 🙏 🫡

As for the Viz:

200px

Screenshot 2025-08-15 at 11 51 30 PM

250px

Screenshot 2025-08-15 at 11 52 06 PM

300px like in @jonfroehlich and team's repo

Screenshot 2025-08-15 at 11 52 18 PM

Cheers

@simonprovost
Copy link
Member Author

UP

@soniacq
Copy link
Contributor

soniacq commented Aug 21, 2025

@simonprovost I would recommend using 200px.

Also, it would be better to use the black version of the logo from the OSCUR webpage (there’s a slight difference in the placement of the end of the circular arrow). Please find the logo attached.

logo

@simonprovost
Copy link
Member Author

I would recommend using 200px.

Also, it would be better to use the black version of the logo from the OSCUR webpage (there’s a slight difference in the placement of the end of the circular arrow). Please find the logo attached.

Very interesting! Did not get access to the drive with the logos but great will include that one in 200px; maybe it's worth tagging @jonfroehlich just in case they would like to unify that too

Thanksss Sonia, will push in a couple of minutes

@simonprovost simonprovost force-pushed the refactor/OSCUR_funding branch from c79149c to 8d30914 Compare August 21, 2025 16:19
@simonprovost
Copy link
Member Author

Done @soniacq :)

For @jonfroehlich , if by any means of interest:

Screenshot 2025-08-21 at 5 17 10 PM

Final Viz:

Screenshot 2025-08-21 at 5 18 36 PM

@soniacq
Copy link
Contributor

soniacq commented Aug 21, 2025

Great! I’ll proceed with the merge once all CI checks have completed successfully.

@soniacq soniacq merged commit 418d8d1 into main Aug 21, 2025
8 of 9 checks passed
@soniacq soniacq deleted the refactor/OSCUR_funding branch August 21, 2025 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants